Skip to content

Update Oracle-to-PostgreSQL Migration Expert: feat: update Oracle-to-Postgres agent model 🤖🤖🤖 - #3364

Closed
PrimedPaul wants to merge 71 commits into
github:mainfrom
PrimedPaul:upstream-promotion/oracle-to-postgres-migration-expert-v1.1.1
Closed

PrimedPaul wants to merge 71 commits into
github:mainfrom
PrimedPaul:upstream-promotion/oracle-to-postgres-migration-expert-v1.1.1

Conversation

@PrimedPaul

@PrimedPaul PrimedPaul commented Sep 19, 2026 •

Copy link
Copy Markdown
Contributor

Summary

This promotion contains the following changes to the Oracle-to-PostgreSQL Migration Expert:

  • feat: update Oracle-to-Postgres agent model

The plugin version is bumped from 1.1.0 to 1.1.1 as part of these changes.

Changed paths

 agents/oracle-to-postgres-migration-expert.agent.md     | 2 +-
 plugins/oracle-to-postgres-migration-expert/plugin.json | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

These changes were developed and reviewed in the fork PrimedPaul/awesome-copilot (main @ 6f5302f8) and bundled onto upstream main @ 4f4796f0. npm run build was run so README and marketplace outputs are in sync.

Checklist

  • Branch created from main
  • npm run build executed
  • bash eng/fix-line-endings.sh executed
  • plugin.json version bumped (1.1.0 → 1.1.1)
  • Maintainer review

Source

Opened by the fork's fork-bundle-upstream-pr workflow (run https://github.com/PrimedPaul/awesome-copilot/actions/runs/35426058576).

GitHub Copilot and others added 30 commits September 11, 2026 13:39
…workflow

- fork-sync-watchdog.yml: Weekly upstream sync + contribution instruction watch
- fork-agent-reviewer.yml: PR-gate with lint + AI domain review (path-scoped)
- fork-bundle-upstream-pr.yml: On-demand upstream PR bundler with cherry-pick promotion
- dev-orchestrator.agent.md: Issue → grill → plan → skeptic review → implement → self-review
- plan-skeptic.agent.md: Adversarial plan critique agent
- contrib-watch.json: State tracking for upstream contribution changes
- upstream-promotion.json: State tracking for fork-to-upstream promotion history
- fork-only/README.md: Complete setup and usage documentation

Implements the complete fork automation architecture with:
- Least-privilege workflows (fine-grained PAT for upstream PRs)
- Weekly sync with upstream (Friday 10:00 AM UTC)
- Path-scoped promotion (agent files only, no fork-specific tooling)
- Plan-mode gating for changes (interactive approval before implementation)
- Skeptical code review before promotion
- Comment-only PR gate (non-blocking advisory feedback)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
- Replace open tasks checklist with completed component inventory
- Document deployed workflows with triggers, scopes, and permissions
- Detail custom agents (dev-orchestrator, plan-skeptic)
- Describe state tracking files and their lifecycle
- Add next steps for user testing and initial usage
- Add sync schedule decision (Friday 10:00 AM UTC)
- Comprehensive status update for reference
…nc and contribution guideline monitoring

- Added a new workflow file for the fork-sync-watchdog that schedules a weekly sync from the upstream repository and provides an AI summary of changes.
- Removed the old fork-sync-watchdog.yml workflow, consolidating functionality into the new markdown-based workflow.
- The new workflow checks for changes in contribution guidelines and workflow files, and opens a pull request if there are updates.
- Introduced structured outputs for guideline changes and new workflows, enhancing the maintainability of the fork.
Fork automation: stateless, PR-based tooling for the Oracle-to-PostgreSQL agent
actions/checkout runs with persist-credentials: false even when a
custom token is supplied, so origin has no stored git credentials.
The later 'git push' step to origin then fails with
'could not read Username for https://github.com'.

Add an explicit step that fails fast if FORK_AUTOMATION_PAT is
missing and rewrites origin's remote URL to embed the token via an
env var, plus sets a bot git identity.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
feat: fork automation workflows + fix sync-watchdog PAT push credentials
…e to watchdog summary

- Push step now emits a targeted ::error:: when the PAT lacks the `workflow`
  scope instead of the raw git refusal (first run failed on this).
- Watchdog agent now recommends Keep / Disable / Integrate for every newly
  added upstream workflow file, with disable steps, and flags modified
  upstream workflows for fork impact.
- README: PAT needs public_repo + workflow; troubleshooting row; weekly-sync
  steps. PLAN: constraint, D18/D19, verification log.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
fix(fork-sync): require workflow scope on PAT; add per-workflow triage to watchdog summary
…and recompile lock file

The review suggestion in #5 added '.github/workflows/*.yaml' to the
workflow-detection globs but was applied with the lines de-indented,
which made the `run:` block invalid YAML, and the .lock.yml was never
recompiled. Fix the indentation and regenerate the lock file.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
fix(fork-sync-watchdog): restore indentation of YAML-glob suggestion and recompile lock file
The agent job fetches upstream via git from github.com, which the gh-aw
firewall blocked. Add the 'github' ecosystem to network.allowed and
recompile the lock file.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Replace the 2-4 sentence prose summary with themed bullets, bold
headlines, section headings, an explicit 'Action needed' line, and
collapsed commit list.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
…-github-domain

fix(fork-sync-watchdog): allow github network egress for agent git fetch
The agent spent the single add_comment quota on a 'test comment' and
then could not post the real summary. Make it explicit that add_comment
is called exactly once with the finished output, and never as a probe.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
…-github-domain

fix(fork-sync-watchdog): forbid test/placeholder add_comment calls
Run logs show the agent completed the analysis and drafted the comment,
then could not post it: gh-aw's own prompt prescribes
'jq -Rs ... | safeoutputs add_comment .' for long bodies, but bash was
allowlisted to 'git *' only, so jq was denied. The fallback
'cat file | safeoutputs add_comment --body -' posted a literal '-' and
exhausted the single add_comment slot.

- Allow 'jq *' in bash tools
- Spell out the exact write-file-then-jq posting command for the
  comment and the issue, and list the transports that must not be used
- Use the schema's 'pr' argument name (pull_request_number is not valid)

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
…-github-domain

fix(fork-sync-watchdog): allow jq so the agent can post the summary body
When the repo owner opens an issue, run the first round of grilling and
planning automatically and hand back a draft PR to work from.

- .github/workflows/fork-issue-planner.md (+ .lock.yml): gh-aw workflow
  triggered by owner-opened issues (fork-automation-labelled issues
  excluded) plus workflow_dispatch for re-runs. It reads the issue and
  the files it touches, classifies scope, and writes a plan containing a
  grilling pass, a concrete implementation plan, a self-adversarial
  skeptic report, open questions and verification commands. Opens a
  draft PR on plan/issue-<N> and comments the open questions on the issue.
- allowed-files restricts the workflow to .github/fork-only/plans/issue-*.md,
  so it can never touch the agent, plugin, skills or any workflow.
- dev-orchestrator now reads the seed plan (and the maintainer's answers)
  in Phase 1 and rewrites it in place; the plan is advisory, not a contract.
- README: architecture diagram, workflow table, development flow,
  PAT scope note, first-run checklist and troubleshooting rows.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
feat(fork-only): plan owner-opened issues into a draft PR
chore(fork-sync): merge upstream main (2026-09-18)
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
docs: record successful fork sync watchdog verification
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
fix: allow planner plan files without review
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
fix: tighten issue planner output contract
Verified via issue #28 -> draft PR #29: correct scope classification,
accurate file reads, full plan sections, one PR + one comment as designed.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
docs(fork-only): mark Fork Issue Planner verification complete
@github-actions

Copy link
Copy Markdown
Contributor

🔒 PR Risk Scan Results

Scanned 3 changed file(s).

Severity Count
🔴 High 0
🟠 Medium 0
ℹ️ Info 0

✅ No matching risk patterns were detected in changed files.

This is an automated soft-gate report. Findings indicate review targets and do not block merge by themselves.

@github-actions

Copy link
Copy Markdown
Contributor

🔍 Vally Lint Results

⚠️ Warnings or advisories found

Scope Checked
Skills 0
Agents 1
Total 1
Severity Count
❌ Errors 0
⚠️ Warnings 0
ℹ️ Advisories 1

Summary

Level Finding
ℹ️ Vally currently lints SKILL.md content. Agent files were detected but skipped:
Full linter output
### Agent files detected (not linted by vally)
ℹ️ Vally currently lints SKILL.md content. Agent files were detected but skipped:
agents/oracle-to-postgres-migration-expert.agent.md

@PrimedPaul
PrimedPaul marked this pull request as ready for review September 19, 2026 06:18
Copilot AI balanced review requested due to automatic review settings September 19, 2026 06:18
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

The configured model name contains an unsupported 5.0 version and must use the canonical Sonnet 5 identifier.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 1 Medium severity

Open (1)
What changed in this PR

Updates the Oracle-to-PostgreSQL migration agent’s model and publishes the change as a patch release.

Changes:

  • Updates the configured AI model.
  • Bumps plugin and marketplace versions to 1.1.1.
File Description
plugins/​oracle-to-postgres-migration-expert/​plugin.json Bumps the plugin version.
agents/​oracle-to-postgres-migration-expert.agent.md Updates the agent model.
.github/​plugin/​marketplace.json Synchronizes generated marketplace metadata.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

PrimedPaul and others added 3 commits September 19, 2026 02:26
chore: remove completed fork automation plan
Updated model name and corrected description formatting.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings September 19, 2026 16:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The supported model change and patch-version metadata are consistent across all affected files.

Review effort: Balanced
Findings: None

Resolved since last review (1)

Copilot AI and others added 7 commits September 19, 2026 22:36
…acle-to-postgres-migration-expert-v1.1.1

# Conflicts:
#	agents/oracle-to-postgres-migration-expert.agent.md

Co-authored-by: PrimedPaul <29710834+PrimedPaul@users.noreply.github.com>
Co-authored-by: PrimedPaul <29710834+PrimedPaul@users.noreply.github.com>
Co-authored-by: PrimedPaul <29710834+PrimedPaul@users.noreply.github.com>
Co-authored-by: PrimedPaul <29710834+PrimedPaul@users.noreply.github.com>
Co-authored-by: PrimedPaul <29710834+PrimedPaul@users.noreply.github.com>
Co-authored-by: PrimedPaul <29710834+PrimedPaul@users.noreply.github.com>
Co-authored-by: PrimedPaul <29710834+PrimedPaul@users.noreply.github.com>
Copilot AI review requested due to automatic review settings September 19, 2026 22:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

The promotion includes fork-only files that must be removed before merging upstream.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 1 High severity

Open (1)

npm run build
git add -A README.md docs .github/plugin/marketplace.json
bash eng/fix-line-endings.sh
git add -A
@PrimedPaul PrimedPaul closed this Sep 22, 2026
@PrimedPaul

Copy link
Copy Markdown
Contributor Author

Closed the PR following leakage from downstream fork. Will investigate and repost.

@PrimedPaul
PrimedPaul deleted the upstream-promotion/oracle-to-postgres-migration-expert-v1.1.1 branch September 24, 2026 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent PR touches agents plugin PR touches plugins

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants